Skip to content

Use shared notification localization - #42

Merged
Pigbibi merged 1 commit into
mainfrom
fix/shared-notification-localization
Apr 22, 2026
Merged

Use shared notification localization#42
Pigbibi merged 1 commit into
mainfrom
fix/shared-notification-localization

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Apr 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • switch duplicated notification localization helpers to the shared QuantPlatformKit helper
  • keep only platform-specific zh replacements where still needed
  • bump QuantPlatformKit to the merged localization helper commit

Testing

  • PYTHONPATH=/tmp/qsl_worktrees/QuantPlatformKit-i18n/src:/tmp/qsl_worktrees/InteractiveBrokersPlatform-i18n /home/ubuntu/Projects/InteractiveBrokersPlatform/.venv/bin/python -m pytest tests/test_rebalance_service.py tests/test_notifications.py -q

@Pigbibi
Pigbibi merged commit a4cc8a2 into main Apr 22, 2026
1 check failed
@Pigbibi
Pigbibi deleted the fix/shared-notification-localization branch May 21, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant